Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(TaskProcessingApiController): Don't use + to merge non-assoc. arrays #46222

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcelklehr
Copy link
Member

Summary

I used + when I should have used array_merge 🙈

Checklist

@marcelklehr marcelklehr force-pushed the fix/task-processing-api-controller/dont-use-plus branch from 2801b24 to 0574d7e Compare July 1, 2024 14:29
@solracsf solracsf added the 3. to review Waiting for reviews label Jul 1, 2024
@solracsf solracsf added this to the Nextcloud 30 milestone Jul 1, 2024
@marcelklehr marcelklehr force-pushed the fix/task-processing-api-controller/dont-use-plus branch from 0574d7e to 224779c Compare July 1, 2024 16:47
This was referenced Jul 30, 2024
This was referenced Aug 5, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 13, 2024
@skjnldsv skjnldsv modified the milestones: Nextcloud 30, Nextcloud 31 Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants